@anlesk any more tweaks to this coming or good to merge?

@philosowaffle sorry, it is not ready yet. I was trying to fix both issues in a single hit, but because GH action runs on docker image that does not have zip installed - the latest addition of pre-zip is failing.
I'll either split in into multiple PRs or fix it within next couple of minutes ;)

I've used this action for zip on other workflows, should work here I believe.

I've used this action for zip on other workflows, should work here I believe.

Interesting.. For some reason, it does not seem to be working as well (https://github.com/anlesk/peloton-to-garmin/actions/runs/6364291199/job/17280467282):

  with:
    files: /app/output/fit
    dest: output.zip
    recursive: true
  env:
    OUTPUT_DIR: /app/output

Ready to zip "/app/output/fit" into output.zip
  - /app/output/fit (Not Found)

Zipped file output.zip successfully

With output.zip being an empty archive file, even though ls -la /app/output/fit clearly shows files in there.

@philosowaffle reverted zip operation back, as it will take more time to fix -- will file new PR.

Really odd it doesn't seem to have the same directory context. I guess could always try running apt-get install zip to manually install

Really odd it doesn't seem to have the same directory context. I guess could always try running apt-get install zip to manually install

Yeah, that does the trick:
https://github.com/anlesk/peloton-to-garmin/actions/runs/6364383784/workflow#L99
https://github.com/anlesk/peloton-to-garmin/actions/runs/6364383784